Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Variables cleanup: PackageVariable #4525

Merged
merged 1 commit into from
May 12, 2024

Conversation

mwichmann
Copy link
Collaborator

@mwichmann mwichmann commented May 10, 2024

Part 3 of a series, updating the PackageVariable implementation, tests and docstrings.

Contributor Checklist:

  • I have created a new test or updated the unit tests to cover the new/changed functionality.
  • I have updated CHANGES.txt (and read the README.rst)
  • I have updated the appropriate documentation

Part 3 of a series, updating the EnumVariable implementation,
tests and docstrings.

Signed-off-by: Mats Wichmann <[email protected]>
@mwichmann mwichmann added maintenance Tasks to maintain internal SCons code/tools Variables Variables() subsystem labels May 10, 2024
@bdbaddog bdbaddog merged commit bcf7158 into SCons:master May 12, 2024
4 of 5 checks passed
@mwichmann mwichmann added this to the 4.8 milestone May 13, 2024
@mwichmann mwichmann deleted the maint/Variables-package branch May 13, 2024 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Tasks to maintain internal SCons code/tools Variables Variables() subsystem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants